video: Stop playing when appropriate
authorMatthias Clasen <mclasen@redhat.com>
Fri, 1 Jan 2021 20:07:25 +0000 (15:07 -0500)
committerMatthias Clasen <mclasen@redhat.com>
Fri, 1 Jan 2021 20:17:59 +0000 (15:17 -0500)
When the widget gets unrealized or hidden, pause
the mediastream. Otherwise, audio might keep playing
unexpectedly.

gtk/gtkvideo.c

index d2cb23321ab5f80f695426aa6227434e86859ee2..5f3293ba64baff54c87b07e49e88314c2ea3cd50 100644 (file)
@@ -131,6 +131,9 @@ gtk_video_unrealize (GtkWidget *widget)
 {
   GtkVideo *self = GTK_VIDEO (widget);
 
+  if (self->autoplay && self->media_stream)
+    gtk_media_stream_pause (self->media_stream);
+
   if (self->media_stream)
     {
       GdkSurface *surface;
@@ -165,11 +168,20 @@ gtk_video_unmap (GtkWidget *widget)
       gtk_revealer_set_reveal_child (GTK_REVEALER (self->controls_revealer), FALSE);
     }
 
-  /* XXX: pause video here? */
-
   GTK_WIDGET_CLASS (gtk_video_parent_class)->unmap (widget);
 }
 
+static void
+gtk_video_hide (GtkWidget *widget)
+{
+  GtkVideo *self = GTK_VIDEO (widget);
+
+  if (self->autoplay && self->media_stream)
+    gtk_media_stream_pause (self->media_stream);
+
+  GTK_WIDGET_CLASS (gtk_video_parent_class)->hide (widget);
+}
+
 static void
 gtk_video_set_focus_child (GtkWidget *widget,
                            GtkWidget *child)
@@ -268,6 +280,7 @@ gtk_video_class_init (GtkVideoClass *klass)
   widget_class->unrealize = gtk_video_unrealize;
   widget_class->map = gtk_video_map;
   widget_class->unmap = gtk_video_unmap;
+  widget_class->hide = gtk_video_hide;
   widget_class->set_focus_child = gtk_video_set_focus_child;
 
   gobject_class->dispose = gtk_video_dispose;